Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add state management access to static functions #72

Closed
wants to merge 1 commit into from

Conversation

markbackman
Copy link
Contributor

@markbackman markbackman commented Dec 29, 2024

Opening this draft with an experiment to make FlowManager state available.

TODOs:

  • Revisit implementation to ensure this is developer friendly
  • Write docs
  • Revisit example for usability
  • Get community feedback

Copy link

vercel bot commented Dec 29, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
pipecat-flows ✅ Ready (Inspect) Visit Preview Dec 29, 2024 4:23pm

@markbackman
Copy link
Contributor Author

I posted this in Discord about this change:

Static flows assume that the JSON is predetermined in advance; this means that your role_messages, task_messages, and function calls are all predefined as well. In a static flow, you can use handlers in your application logic for dynamic state.

If you require dynamic changes of the role_messages, task_messages and function calls, then the dynamic flow is the way to go.

I worked with someone at Daily through a static flow demo yesterday and kind of soured on the idea I proposed in the PR. I think I'm going to keep static flows truly static for now. If you need dynamic state, then dynamic flows are an option.

Closing this for now. If you need to dynamically change the flow, then I recommend using the dynamic flows.

@markbackman markbackman closed this Jan 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant